Skip to content

Added support for Next Available Subnet by Tags in Terraform#202

Merged
unasra merged 15 commits into
infobloxopen:masterfrom
Tejashree-RS:na_subnet_by_tags
Jun 19, 2025
Merged

Added support for Next Available Subnet by Tags in Terraform#202
unasra merged 15 commits into
infobloxopen:masterfrom
Tejashree-RS:na_subnet_by_tags

Conversation

@Tejashree-RS

Copy link
Copy Markdown
Collaborator

No description provided.

@JchhatbarInfoblox JchhatbarInfoblox left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds optional tag_filters support to the Next Available Subnet Terraform data source, enabling users to filter address blocks by tags.

  • Introduces a tag_filters schema attribute and implements logic to list and filter subnets by matching tags.
  • Adds helper functions (buildTagFilterString, error parsing, findSubnet) and updates error handling/ retry behavior.
  • Updates tests, examples, and documentation to cover and illustrate the new tag_filters feature.

Reviewed Changes

Copilot reviewed 23 out of 23 changed files in this pull request and generated 5 comments.

File Description
internal/service/ipam/api_next_available_subnet_data_source.go Schema updates, tag filter logic, retry & error parsing
internal/service/ipam/api_next_available_subnet_data_source_test.go New acceptance tests for single/multiple tag filters
examples/data-sources/bloxone_ipam_next_available_subnets/data-source.tf Example usage with tag_filters
docs/data-sources/ipam_next_available_subnets.md Documentation updates for tag_filters and examples

Comment thread internal/service/ipam/api_next_available_subnet_data_source.go Outdated
Comment thread internal/service/ipam/api_next_available_subnet_data_source.go Outdated
Comment thread internal/service/ipam/api_next_available_subnet_data_source.go Outdated
Comment thread docs/data-sources/ipam_next_available_subnets.md
Comment thread internal/service/ipam/api_next_available_subnet_data_source.go

@unasra unasra left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Relevant changes for this PR have been mentioned in the Addressblock PR

@Tejashree-RS Tejashree-RS requested a review from unasra June 9, 2025 06:11
Comment thread internal/service/ipam/api_next_available_subnet_data_source.go Outdated
@unasra unasra merged commit 4111bac into infobloxopen:master Jun 19, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants